Repository navigation
refactor(api-docs): remove redux and split into docs, playground and shared packages - #128
Open
arpit-bruno wants to merge 73 commits into
Open
arpit-bruno wants to merge 73 commits into
arpit-bruno wants to merge 73 commits into
Conversation
…n, which persists it
…store subscribe pump
…opencollection and pass git url and source text to appshell as props, delete the app slice
… so a folder toggle no longer rebuilds the nav model, and build the uuid to slug map once in the nav model
…ook in opencollection that owns the loaded document
…th its nav model through a document context, delete the docs slice
… collapse across close, reopen, dock switch and updatecollection
…servable and derive the view from it
…e per-instance session so they still survive close
… session instead of isCollapsed on the items, carried over on re-seed as before
…s views selected, as before
…eir tests, not wired yet
…callbacks instead of dispatching, still backed by redux
…responses and run starts; delete the playground slice and the now empty store
… loading the bundle cross-origin without cors
…t of runner and playground so shared code stops importing them
…itest and typecheck, run tests and typecheck across workspaces in ci
…s from env label specs and the playground header comment
… and wrap their long lines
…t used any and wrap long lines
…them as @usebruno/api-shared from docs
…und and lazy-load it from the docs dock shell as @usebruno/api-playground
…nd, playground never imports docs, and docs only type-imports the playground
…t reads react, router and the shared context modules from Bruno.__rt
…yground file on first open instead of bundling it into api-docs.js
…ep link, once per page, and that a missing file shows the load error
…y, the playground cdn load and the builder cache requirement, and repoint every moved path
…d file stops bundling a second prism that rewrites docs snippets on open
…d of caching the rejection until reload
…so a nonce-only csp allows it
…wo api-docs copies sharing globals, and the amd currentScript fallback
…and lint every package against its own package.json
…ime hands the playground every one of them
… a failed load so the next open retries
…dn works on a fresh tree
arpit-bruno
changed the title
arpit-bruno
changed the title
arpit-bruno
changed the title
arpit-bruno
added
run-integrations-matrix-tests
arpit-bruno
marked this pull request as ready for review
November 13, 2026 00:15
… ci checkout can typecheck
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@coderabbitai ignore
Ref BRU-4102
Do not merge or deploy until the builder change is live (hashed
api-playground-*.jsimmutable and never deleted,api-docs.*short cache). Otherwise browsers with a cachedapi-docs.jsask for a deleted playground file.Redux removal
@reduxjs/toolkit,react-redux,src/store, the@slicesalias)useSyncExternalStoreinstead of a store, no new dependencySplit into three packages (BRU-4102)
@usebruno/api-docs: docs view, dock shell, CDN entry, e2e@usebruno/api-playground(private): playground, request runner, scripting@usebruno/api-shared(private): session, document, theme, shared UI and helperspackage.json; 7 unused deps removedapi-docs.js8.5 MB → 4.8 MB; the playground loads on first "Try it" asapi-playground-<hash>.js, once per pageBruno.apiDocs,window.OpenCollection, CSS and entry file names unchanged@/alias now only in docs; shared and playground use package-name importshidePlayground: true(the playground file is never downloaded); playground only is not part of this PR (BRU-4789)Tests
updateCollection/updateOptions, loading inputs, env after reload, two instances, what survives close/reopenhidePlaygroundStandards
CODING_STANDARDS.md,ARCHITECTURE.md,AGENTS.md, rootCLAUDE.md(importsAGENTS.md),.coderabbit.yamlno-explicit-anyis an error on changed lines; file naming rule (PascalCase components, camelCase rest, kebab e2e)contributing.md: PRs target the activerelease/vX.Y.0branchChecked